Skip to content

Add SPEC.md documenting all current features#48

Merged
asmacdo merged 4 commits intomainfrom
add-spec
Apr 4, 2026
Merged

Add SPEC.md documenting all current features#48
asmacdo merged 4 commits intomainfrom
add-spec

Conversation

@asmacdo
Copy link
Copy Markdown
Member

@asmacdo asmacdo commented Mar 12, 2026

Addresses #42. Extracts a comprehensive specification from the current implementation covering CLI, configuration, volumes, path modes, worktree support, NVIDIA GPU, container runtime, Dockerfile, setup script, security boundaries, testing, and CI/CD.

Produced by Claude-code

Addresses #42. Extracts a comprehensive specification from the current
implementation covering CLI, configuration, volumes, path modes, worktree
support, NVIDIA GPU, container runtime, Dockerfile, setup script, security
boundaries, testing, and CI/CD.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
SPEC.md Outdated

### Triggers

- Push to `main` or `enhs` branches.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

while at it - purge that outdated and unused enhs from branches, CI and here

Suggested change
- Push to `main` or `enhs` branches.
- Push to `main` branches.

SPEC.md Outdated

### Default Mounts

| Mount | Host Path | Container Path | Options |
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ask claude to reformat all tables to be also for human consumption (properly indented)

@@ -0,0 +1,440 @@
# YOLO Specification
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

add to CLAUDE.md a strong note to consult SPEC.md file for any work on yolo

SPEC.md Outdated
# YOLO Specification

Extracted from the current implementation as of 2026-03-12.
See [issue #42](https://github.com/con/yolo/issues/42).
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

information not useful long run

Suggested change
See [issue #42](https://github.com/con/yolo/issues/42).

- Reformat all SPEC.md tables with proper column alignment for readability
- Remove dated "Extracted from..." header (not useful long-term)
- Add CLAUDE.md with directive to consult SPEC.md for yolo work
- Remove unused `enhs` branch from CI triggers and spec

Co-Authored-By: Yaroslav Halchenko <debian@onerussian.com>
Co-Authored-By: Austin Macdonald <austin@dartmouth.edu>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
CLAUDE.md Outdated
@@ -0,0 +1,3 @@
# YOLO Development

**IMPORTANT: Always consult `SPEC.md` before making any changes to yolo.** It documents all current features, CLI flags, configuration, container runtime behavior, and security boundaries. Any modifications must stay consistent with the spec, and the spec must be updated to reflect any changes.
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

well, if to have this, you should also instruct here to here to reflect any changes in YOLO behavior and interfaces etc mentioned in the file in that file happen they change. Otherwise this duplicated information will diverge soon! Eg. claude happily coded me #53 without changing anything in the spec, although likely he should have

@yarikoptic yarikoptic marked this pull request as draft April 1, 2026 16:06
asmacdo and others added 2 commits April 4, 2026 15:09
- SELinux labels: :Z → :z (shared) for all default mounts (issue #51)
- UID mapping: remove --user, update --userns to keep-id:uid=1000,gid=1000 (PR #61)
- Add deno extra to build args, extras table, setup flags, PATH (PR #56)
- Document container name reuse as claude --name session name
- CI triggers: include enhs branch

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@asmacdo asmacdo marked this pull request as ready for review April 4, 2026 20:16
@asmacdo
Copy link
Copy Markdown
Member Author

asmacdo commented Apr 4, 2026

I've asked claude to update the spec, to current main.

I need this merged for an experiment im running ;)
IMO since this is just adding 2 markdown files, I am going to merge now, we can iterate on top. Better to get it in before more features so spec cna be updated by the humans/agents making the changes.

@asmacdo asmacdo merged commit 7acaa54 into main Apr 4, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants